Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Replace protobuf snapshot batch #196

Merged
merged 5 commits into from
Feb 21, 2024
Merged

Replace protobuf snapshot batch #196

merged 5 commits into from
Feb 21, 2024

Conversation

JackyWoo
Copy link
Contributor

@JackyWoo JackyWoo commented Feb 20, 2024 •
edited
Loading

Which issues of this PR fixes:

This PR try to fix #192

Change log:

  • Replace protobuf snapshot batch
  • Fix error when parse snapshot acl of v0

@JackyWoo JackyWoo added the enhancement Enhancements label Feb 20, 2024
@JackyWoo JackyWoo added this to the Release v2.0.4 milestone Feb 20, 2024
@JackyWoo JackyWoo force-pushed the replace_protobuf_snapshot_batch branch from 1f99fa5 to 86edd35 Compare February 20, 2024 06:51
@JackyWoo JackyWoo force-pushed the replace_protobuf_snapshot_batch branch from 3badd6b to 7d7aaba Compare February 20, 2024 06:57
@JackyWoo JackyWoo mentioned this pull request Feb 20, 2024
Closed
2 tasks
@lzydmxy
Copy link
Contributor

lzydmxy commented Feb 21, 2024

LGTM

@JackyWoo JackyWoo merged commit 902b0ae into JDRaftKeeper:master Feb 21, 2024
6 checks passed
@nicelulu nicelulu added enhancement Enhancements and removed enhancement Enhancements labels Feb 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lzydmxy lzydmxy lzydmxy left review comments

Assignees
No one assigned
Labels
enhancement Enhancements
Projects
None yet
Milestone
Release v2.0.4
Development

Successfully merging this pull request may close these issues.

Remove dependency of third party lib protobuf.
3 participants
@JackyWoo @lzydmxy @nicelulu